feat: add helm history tool #423
Conversation
iamsudip
force-pushed
the
main
branch
2 times, most recently
from
684b67a to
64fd609
Compare
Signed-off-by: iamsudip <[email protected]> Signed-off-by: iamsudip <[email protected]>
Signed-off-by: iamsudip <[email protected]>
| if len(releases) == 0 { | ||
| return fmt.Sprintf("No history found for release %s", name), nil | ||
| } | ||
| if max > 0 && len(releases) > max { |
There was a problem hiding this comment.
I had to do it manually here cause helm cli also does same, the history object has a field Max, but that is still not used in history.Run() method to limit the result. So we fetch all, limit on client side to mimic the same behaviour of cli.
|
@manusa @ardaguclu Please review! |
Don't worry, we'll look into it. One of the things we want to consider is if we want to keep having helm as part of the default toolsets or make it opt-in instead. |
|
Actually I did and it generated some extra docs, I see which you have already merged. I will rebase the branch and re-run. Right now it's part of helm toolset, are you looking to make helm toolset optional in future? |
|
@manusa I reran |
2 participants
No description provided.